Skip to content
This repository was archived by the owner on Apr 11, 2024. It is now read-only.

Add support for building and bundling windows binaries.#117

Open
matthewhartstonge wants to merge 7 commits intomercari:masterfrom
matthewhartstonge:feature/windows-support
Open

Add support for building and bundling windows binaries.#117
matthewhartstonge wants to merge 7 commits intomercari:masterfrom
matthewhartstonge:feature/windows-support

Conversation

@matthewhartstonge
Copy link

@matthewhartstonge matthewhartstonge commented May 9, 2019

As a first step towards #116 I've added windows support to the makefile for creating binaries.

@matthewhartstonge matthewhartstonge changed the title make: adds support for building and bundling windows binaries. wip: make: adds support for building and bundling windows binaries. May 9, 2019
@matthewhartstonge
Copy link
Author

matthewhartstonge commented May 9, 2019

currently there is one test failing under windows, I'll have a look into this.

=== RUN   TestGetListener
--- FAIL: TestGetListener (0.00s)
        Error Trace:    server_test.go:47
        Error:          Expected nil, but got: &errors.errorString{s:"failed to listen unix socket: listen unix /tmp/gaurun.sock: bind: A socket operation encountered a dead network."}

@matthewhartstonge matthewhartstonge changed the title wip: make: adds support for building and bundling windows binaries. Add support for building and bundling windows binaries. Oct 15, 2020
@matthewhartstonge
Copy link
Author

matthewhartstonge commented Oct 15, 2020

Looks like the p12 certificate has expired under gaurun/buford/testdata/cert.p12

@cubicdaiya
Copy link
Contributor

cubicdaiya commented Feb 26, 2021 •
edited
Loading

Since gaurun does not use p12 in actual, we removed the code to handle p12 in #146.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@matthewhartstonge @cubicdaiya